feat!: Drop the legacy studio course home page #37454
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
feanil
force-pushed
the
feanil/drop_course_home
branch
3 times, most recently
from
eca64c8 to
b28cc87
Compare
52 tasks
feanil
force-pushed
the
feanil/drop_course_home
branch
from
6880a44 to
37dfbc4
Compare
feanil
force-pushed
the
feanil/drop_legacy_editor_config
branch
from
9f406cc to
227e818
Compare
feanil
force-pushed
the
feanil/drop_course_home
branch
from
37dfbc4 to
7057510
Compare
feanil
force-pushed
the
feanil/drop_legacy_editor_config
branch
from
7c78f45 to
6ac4201
Compare
feanil
force-pushed
the
feanil/drop_course_home
branch
2 times, most recently
from
ade2299 to
4375bf5
Compare
kdmccormick
reviewed
Oct 16, 2025
cms/djangoapps/contentstore/rest_api/v1/serializers/course_waffle_flags.py
Show resolved
Hide resolved
This is the page that lists the courses in studio. This has been replaced by an MFE and the MFE has been on by default since Teak. BREAKING CHANGE: Setting the `legacy_studio.home` waffle flag will no longer work. The code will behave as if this is set to false showing the new studio authoring MFE experience for the course home page. This has been the default since Teak.
This view has been replaced with a new courses API endpoint `/contentstore/v2/home/courses` which has its own tests.
We had a set of i18n tests that testing basic english is working on a page that's going away. It's not necessary. And then we have a skipped test that tests i18n. It would be good to have actual i18n tests and there are some other ones in this file at the block level that do test i18n but since this test isn't actually testing anything novel there's no need to keep it.
Since we're trying to drop the underlying page, switch to the equivalent API views for these contentstore tests.
The url previously pointed to the legacy index.html with the libraries tab loaded. Now it redirects to the new libraries endpoint in the authoring MFE.
feanil
force-pushed
the
feanil/drop_course_home
branch
from
4375bf5 to
7c8020d
Compare
kdmccormick
approved these changes
Oct 17, 2025
Member
kdmccormick
left a comment
kdmccormick
left a comment
There was a problem hiding this comment.
Two nits:
- link the followup in the code (suggestion below)
- reword the commit to "feat!: Drop the legacy studio
coursehome page" . This PR drops the general studio homepage (the course listing) but the commit message makes it sound like it's dropping the course-specific homepage (the outline)
cms/djangoapps/contentstore/rest_api/v1/serializers/course_waffle_flags.py
Show resolved
Hide resolved
kdmccormick
requested changes
Oct 17, 2025
| """ | ||
| List all Libraries available to the logged in user | ||
| """ | ||
| data = get_library_context(request) |
Member
There was a problem hiding this comment.
ack, we might need to keep the legacy libraries home around for Ulmo, let me check on that before you merge.
kdmccormick
approved these changes
Oct 17, 2025
Co-authored-by: Kyle McCormick <kyle@axim.org>
feanil
force-pushed
the
feanil/drop_course_home
branch
from
df7bd42 to
1ca24ee
Compare
7 tasks
kdmccormick
added a commit
to openedx/frontend-app-authoring
that referenced
this pull request
Oct 22, 2025
This adds a CreateLegacyLibrary component. It functions the same as CreateLibrary, but it calls the V1 (legacy) creation REST API rather the V2 (new/beta) REST API. This reinstates, in the MFE, something that was possible using the legacy frontend until it was prematurely removed by openedx/openedx-platform#37454. We plan to re-remove this ability between Ulmo and Verawood as part of: openedx/openedx-platform#32457. So, we have intentionally avoided factoring out common logic between CreateLibrary and CreateLegacyLibrary, ensuring that the latter remains easy to remove and clean up.
Faraz32123
pushed a commit
to edx/frontend-app-authoring
that referenced
this pull request
Oct 24, 2025
This adds a CreateLegacyLibrary component. It functions the same as CreateLibrary, but it calls the V1 (legacy) creation REST API rather the V2 (new/beta) REST API. This reinstates, in the MFE, something that was possible using the legacy frontend until it was prematurely removed by openedx/openedx-platform#37454. We plan to re-remove this ability between Ulmo and Verawood as part of: openedx/openedx-platform#32457. So, we have intentionally avoided factoring out common logic between CreateLibrary and CreateLegacyLibrary, ensuring that the latter remains easy to remove and clean up.
Faraz32123
pushed a commit
to edx/frontend-app-authoring
that referenced
this pull request
Oct 24, 2025
This adds a CreateLegacyLibrary component. It functions the same as CreateLibrary, but it calls the V1 (legacy) creation REST API rather the V2 (new/beta) REST API. This reinstates, in the MFE, something that was possible using the legacy frontend until it was prematurely removed by openedx/openedx-platform#37454. We plan to re-remove this ability between Ulmo and Verawood as part of: openedx/openedx-platform#32457. So, we have intentionally avoided factoring out common logic between CreateLibrary and CreateLegacyLibrary, ensuring that the latter remains easy to remove and clean up.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Part of #36108
The Studio Home page has been replaced for some time. This is code that
drops the ability to fall back to the old experience. The old experience
has not been the default since Teak (Apr 2025).
The waffle flag being removed (
toggles.LEGACY_STUDIO_HOME) is still passed through to an API endpoint, so there is likely frontend changes than need to be done in the future once all the edx-platoform cleanup is completed. This PR drops all the items that are safe to drop without changing interfaces with other services. The work to drop the underlying API that controls which services to use will be done as follow up work in #37497That allows this work to be completed without coupling it to updates upstream.